Skip to content

feat(login): redirect when Router::fullBaseUrl() is not same as App.loginUrl#65

Merged
watura merged 3 commits intoNetCommons3:availabilityfrom
WillBooster:wtr/login-redirect
Jan 8, 2020
Merged

feat(login): redirect when Router::fullBaseUrl() is not same as App.loginUrl#65
watura merged 3 commits intoNetCommons3:availabilityfrom
WillBooster:wtr/login-redirect

Conversation

@watura
Copy link

@watura watura commented Dec 22, 2019

やったこと

application.yml に App.loginUrl を定義した状態で、

なお、楽に確認するためには NetCommons3/NetCommons#561 とかがはいっているといいです。
example.com.ymlsecure.example.com.yml を用意して、example.com.yml には、fullBaseUrl と loginUrlを定義すると、テストがやりやすくなります。

なぜやるか

CDNで http://example.com はキャッシュして、http://secure.example.com はキャッシュしない。
でも、http://example.com から、利用者はほぼ考えることなく、 http://secure.example.com に遷移できるようにしたい。
ログイン画面に遷移しようとした時に、自動的に遷移させるようにすると、利用者は考えなくても遷移できているはずだ。と言う考えから実装しました。

もっとNC3の深いところから、secureとnon-secureを分けて動作するようにするのもよいかと思ったんですが、影響範囲が大きくなって、テストが難しくなるので、シンプルに分けるようにしました。

レビュー観点

  • loginUrl と言うコンフィグ名の妥当性
    • もっとなんかいい名前を考えたいです

@watura watura requested a review from exKAZUu December 22, 2019 23:48
@coveralls
Copy link

coveralls commented Dec 23, 2019

Coverage Status

Coverage increased (+0.3%) to 38.545% when pulling 4766b5d on WillBooster:wtr/login-redirect into cfd9849 on NetCommons3:availability.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 38.962% when pulling 4766b5d on WillBooster:wtr/login-redirect into cfd9849 on NetCommons3:availability.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 38.962% when pulling 4766b5d on WillBooster:wtr/login-redirect into cfd9849 on NetCommons3:availability.

@watura watura merged commit 3319089 into NetCommons3:availability Jan 8, 2020
@watura watura deleted the wtr/login-redirect branch January 8, 2020 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants